-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding poro howto #1673
Adding poro howto #1673
Conversation
@id = attributes[:id] | ||
@name = attributes[:name] | ||
@level = attributes[:level] | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you fix the indentation on these lines?
hey, can you add a link to your new doc under and then add a changelog entry to give yourself credit? :-) |
@NullVoxPopuli can you tell me how to do that? |
trying to get styling correct added changelog entry, link to guide, and fixed indentation
yup, I have some instructions here: https://github.com/NullVoxPopuli/git-workflow/blob/master/single-repo/README.md#preparing-your-feature--bug-fix-to-be-merged lemme know if you have any questions |
d9f0d61
to
05618ea
Compare
@NullVoxPopuli let me know if I did it correctly |
it looks like you did! |
LGTM! thanks, @DrSayre |
oh, it looks like I could have squashed. It's after I click merge. oops. |
no worries, glad I could help out! |
Purpose
Easier to find documentation on using AMS with a PORO
Changes
Adds a guide under how to showing how to create a PORO that can be used with AMS.
Caveats
Related GitHub issues
#1521
Additional helpful information